Closed
Conversation
keith-turner
reviewed
Apr 1, 2026
| <artifactId>opentelemetry-proto</artifactId> | ||
| <!-- This version was selected because it aligns with the version of protocol buffers that Accumulo 2.1 is using. --> | ||
| <version>1.3.2-alpha</version> | ||
| <version>1.10.0-alpha</version> |
Contributor
There was a problem hiding this comment.
Changing this version causes ScanTracingIT to fail because it sets up the agent, the agent uses this library which uses protobuf. The protobuf version this library depends on differs from the protobuf version accumulo depends on.
When I run the test w/ these changes I see exceptions like the following. Not sure if this just at test problem or if this dep change will cause server side serialization of trace data to fail.
java.lang.NoClassDefFoundError: com/google/protobuf/RuntimeVersion$RuntimeDomain
at io.opentelemetry.proto.collector.trace.v1.ExportTraceServiceRequest.<clinit>(ExportTraceServiceRequest.java:18)
at org.apache.accumulo.test.tracing.TraceCollector$TraceHandler.handle(TraceCollector.java:60)
at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:122)
at org.eclipse.jetty.server.Server.handle(Server.java:563)
at org.eclipse.jetty.server.HttpChannel$RequestDispatchable.dispatch(HttpChannel.java:1598)
at org.eclipse.jetty.server.HttpChannel.dispatch(HttpChannel.java:753)
at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:501)
at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:287)
at org.eclipse.jetty.io.AbstractConnection$ReadCallback.succeeded(AbstractConnection.java:314)
at org.eclipse.jetty.io.FillInterest.fillable(FillInterest.java:100)
at org.eclipse.jetty.io.SelectableChannelEndPoint$1.run(SelectableChannelEndPoint.java:53)
at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:969)
at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.doRunJob(QueuedThreadPool.java:1194)
at org.eclipse.jetty.util.thread.QueuedThreadPool$Runner.run(QueuedThreadPool.java:1149)
at java.base/java.lang.Thread.run(Thread.java:1583)
Caused by: java.lang.ClassNotFoundException: com.google.protobuf.RuntimeVersion$RuntimeDomain
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:641)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:188)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:526)
... 15 more
Contributor
Author
There was a problem hiding this comment.
Good information, I didn't now we had an IT for tracing. I'll look at this.
Contributor
Author
There was a problem hiding this comment.
I had to revert the bom and proto dependencies to get the test to pass. I think I'm going to close this and see if we can address this another way.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade OpenTelemetry dependencies. This should resolve https://github.com/apache/accumulo/security/dependabot/25, but
tracing will need to be tested before release.